Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added analog metrics. #40

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

WattTheFlux
Copy link

Added in tracking the analog distance for the joystick in addition to computing the distance traveled in meters and the average velocity of the stick throughout the game. Analog distance is measured in joystick coordinate units while meters and meters per second are used for the real world distances. After looking at the average velocity, it's not as interesting as a metric as I initially thought it'd be. It's pretty low just due to the fact that in general, more frames are spent holding a direction than moving it. Maybe it'd be more interesting to track joystick acceleration or to exclude frames that don't have any movement? Eh.

Joystick distance traveled in stick units in addition to the average stick velocity.
Copy link
Member

@NikhilNarayana NikhilNarayana left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good functionality wise, obviously the one question mark is size of the stick gate. If we could reference some authority, that would help me feel more confident.

src/stats/common.ts Outdated Show resolved Hide resolved
AnalogMotionType contains the previously top level variables in addition to the newer "analogMotionFrameCount". This is used to track how fast the joystick was moving when it was in motion. It feels like a more meaningful metric to me in conjunction with the frames spent in motion metric.
Uh, that was weird. Not sure how that line got deleted between building and committing...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants